Skip to content

Beta fixes #820

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 26 commits into from
Feb 8, 2025
Merged

Beta fixes #820

merged 26 commits into from
Feb 8, 2025

Conversation

digimer
Copy link
Member

@digimer digimer commented Jan 29, 2025

No description provided.

This was referenced Jan 30, 2025
Open
Open
@digimer-bot
Moved System->check_firewall() to Network.
0db0f0c 
* Also started work on letting the user set an override for the default
  zone.

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Removing the --machine check when clearing jobs.
891bffd 
Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Fixed XML generation.
fd01c5e 
Resolves issue #656

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Changed a job update to print to log also.
188f256 
Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Improved handling of servers that can't be stopped.
14611d4 
Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Started work on System->check_ntp().
66865ee 
Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Finished adding custom NTP server support.
1531250 
* Addresses issue #816

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Updated anvil-manage-server-storage to take --ci-test
d3e076c 
* Created System->_abort_if_ci() to exit if the --ci-test switch is in
  use.

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Updated Server->locate() to not try to connect to hosts that are not
d5b7985 
online. This prevents long delays when hosts are not accessible.

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Updated Database->get_lvm_data() to not load deleted LVM data.
c030ba9 
This resolves issue #657.

Signed-off-by: Madison Kelly <[email protected]>
@digimer
Copy link
Member Author

digimer commented Feb 1, 2025

retest this please

@fabbione fabbione mentioned this pull request Feb 2, 2025
Open
@digimer-bot
Updated System->check_ntp() to use pool instead of Server
13c752b 
It also now comments out instead of removes unused Servers or pools.

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Added checks to avoid 'unitialized value' messages for not-yet-created
0acae19 
DRBD resources.

Address comment #657 (comment)

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Fixed a bug where a request to display information was trying to assign
75469ec 
a job to the host.

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Made all job update prints conditional on if --machine is used.
79578c1 
Addresses regression mentioned in #656 (comment)

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Fixed a bad regex
b3b54ca 
Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Fixed server count issue raised in issue #824
1decde4 
Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Fixed bad strings (missing context and wrong keys).
838f35a 
* Addresses issue #827

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Updated anvil-join-anvil to use System->check_ntp().
6a5f2a3 
* TBD, but should improve issue #668

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Updated Network->manage_firewall({task => 'check'}) to update the
b90b612 
default zone to IFN1 if it's 'default' and IFN1 exists.

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Updated Network->manage_firewall to honour sys::firewall::default-zone
76a98e1 
if set.

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Updated Network->manage_firewall to use firewall-cmd instead of editing
80df15a 
firewalld.conf

Signed-off-by: Madison Kelly <[email protected]>
@digimer-bot
Updated Network->manage_firewall() to assign all interfaces to the
9337c5c 
appropriate zones, not just interfaces with IPs.

Signed-off-by: Madison Kelly <[email protected]>
@digimer digimer mentioned this pull request Feb 6, 2025
Open
@digimer
Copy link
Member Author

digimer commented Feb 6, 2025

retest this please

1 similar comment
@digimer
Copy link
Member Author

digimer commented Feb 6, 2025

retest this please

@ylei-tsubame ylei-tsubame merged commit e8d21f8 into main Feb 8, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ylei-tsubame ylei-tsubame ylei-tsubame approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@digimer @ylei-tsubame @digimer-bot